Skip to content

fix kornia pyramid padding import for compatibility with v0.8.3+#506

Open
DevNullInc wants to merge 1 commit into
Lightricks:masterfrom
DevNullInc:patch-2
Open

fix kornia pyramid padding import for compatibility with v0.8.3+#506
DevNullInc wants to merge 1 commit into
Lightricks:masterfrom
DevNullInc:patch-2

Conversation

@DevNullInc

Copy link
Copy Markdown

The latest updates to the kornia library totally broke this file because they removed the pad function from the pyramid submodule. Because of that, it was throwing a fatal import error and causing the entire extension to fail silently on launch whenever ComfyUI booted up. This patch fixes the crash by pulling pad out of the kornia list and just mapping it directly to the torch functional pad that is already loaded in the environment. It keeps everything working exactly the same way but actually lets the node pack load properly on newer setups.

The latest updates to the kornia library totally broke this file because they removed the pad function from the pyramid submodule. Because of that, it was throwing a fatal import error and causing the entire extension to fail silently on launch whenever ComfyUI booted up. This patch fixes the crash by pulling pad out of the kornia list and just mapping it directly to the torch functional pad that is already loaded in the environment. It keeps everything working exactly the same way but actually lets the node pack load properly on newer setups.

@DevNullInc DevNullInc left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Referencing issue #505
This fixes that error without the need to downgrade Kornia.

This was referenced Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant